Fix Kikongo translation of default:mm

Fix Kikongo translation of default:LTR

Fix Bosnian translation of default:mm

gtkbookmarksmanager: don't allow non valid utf8 in bookmarks

In case some client send to us a non valid utf8 string, don't screw up
the bookmarks file and just return.

https://bugzilla.gnome.org/show_bug.cgi?id=755215

gtkplacessidebar: avoid to use a freed string

The string we were using is the representation of the internal text
in the popover entry. However that can be freed before setting the
bookmark label, if i.e. the row is destroyed and therefore the popover
as well.
To avoid that, duplicate the label in a local variable.

One of the consequences is that for those people using development version
we migth screwed its bookmarks file, since the bookmark manager wrote
garbage from the already freed label.

https://bugzilla.gnome.org/show_bug.cgi?id=755215

build/Makefile-newvs.am: Update Comments for Usage

Update the notes that this is also used for Visual Studio 2015 support,
and correct the MSVC_VER_LONG for MSVC 2015, which is 14, not 2015.

Also add a note that this can be used for other projects that have
Visual Studio build support.

Build: Make Makefile.msvcproj A Bit More Generic

Handle also the situation where $(srcdir) == $(top_srcdir), so that this
can also be used in cases like librsvg and gobject-introspection.

textview: Look up the drop location iter on the right coordinates on DND

We are passing widget coordinates to gtk_text_view_window_to_buffer_coords()
which expects coordinates to be relative to the text window in this case.
This may result in drop coordinates being displaced if the side windows to
the top/left sides are visible and taking space, so the DnD indicator will
point to the wrong position.

This can be seen on gnome-builder and gedit when displaying line numbers.

Updated German translation

Don't mix in user options of printer instances

GTK+ currently does not support printer instances in the printing
dialog. This commit avoids user options set for specific instances of
a printer being accidently applied to the printer with the same name but
no specific instance defined.

https://bugzilla.gnome.org/show_bug.cgi?id=753628
https://bugzilla.gnome.org/show_bug.cgi?id=582747

build: Dist toarray.pl

Some distributions (MSYS2, Debian) call autoreconf on a tarball because
they change the autotools artifacts.

In order to rebuild the Broadway generated files, we need to ship the
scripts that we use when disting a release.

HC: use round buttons on sidebars

Copy Adwaita in how sidebar icon buttons look

https://bugzilla.gnome.org/show_bug.cgi?id=755268

gtkplacessidebar: avoid accessing destroyed popover

The row and rename popovers are always relative_to a row.
We also keep a pointer to them so we can interact with them in
callbacks.

However, if the row is destroyed its associated popovers will be
destroyed as well as relative_to destroyes and frees memory of its
associated widget when its relative_to widget is destroyed.
If we, for example, update the places while the popover is shown we are
going to access and invalid widget on the next time.

To avoid that, connect to the destroy signal of the popovers and clean
the sidebar pointers when that happens.

https://bugzilla.gnome.org/show_bug.cgi?id=755444

GtkWindow: Check for popover realized-ness before unrealizing

https://bugzilla.gnome.org/show_bug.cgi?id=755473

gtkmain: Add nullable annotations

https://bugzilla.gnome.org/show_bug.cgi?id=755038

Update Arabic translation

application: Be save against impl being NULL

We clear the impl in shutdown, so all callbacks that may
happen at a later time need to be prepared for this.

https://bugzilla.gnome.org/show_bug.cgi?id=755432

Fix make dist

Commit ea241c6 dropped the wrong private header from dist, so correct
this to fix 'make dist'

Fix Visual Studio Build Since a080cb4

The patch did not check for Visual Studio 2008 correctly, plus it
would break the build on later Visual Studio versions, as it should
be __popcnt(), not __popcount().  Fix that.

Improve compiler detection for __builtin_popcount()

The popcount builtin was added in GCC after version 4.2 (which is what
some *BSDs are using), which means we need to be more specific when
using it than just asking for GCC.

While we're at it, we can improve the compiler detection, and use a
builtin popcount on Clang ≥ 3.1 and MSVC 2008.

https://bugzilla.gnome.org/show_bug.cgi?id=755455

Use stupid quotes instead of dumb quotes

Following a similar change in GLib a while ago.

'bla' may by stupid, but it looks less dumb than `bla'.

container: Use qdata for focus chain

widget: Add a few more inlined getters

size group: Streamline disposal

Remove widgets from their size groups in dispose, without connecting
to ::destroy. The signal handler is not necessary.

size request: Avoid unnecessary reffing

There is no particular reason why we should have to ref the
widgets if they are in a size group - we are not reffing them
in the simple case either.

size group: streamline iteration slightly

Instead of handling the horizontal and vertical peers separately
(and often, duplicatively), collect all peers in one go. At the
same time, avoid creating and destroying hash tables more often
than necessary.

inspector: Ellipsize the accessible name

It can be long, and we don't want the inspector to grow excessively.

text view: Use qdata with preallocated quarks

entry: Use qdata with preallocated quarks

label: Avoid recalculation in some cases

Avoid recalculation the entire label layout when nothing changed.

scale: Drop unneeded private exports

Nobody outside gtkscale.c is using these functions, so make
them static.

assistant: Drop unneeded calls

GtkAssistant is just a regular composite widget nowadays,
no need to manually queue resizes.

style context: Trivial refactor

Use g_set_object for setting the frame clock.

build/win32: Fix 'make -jN dist'

The recent changes to build/win32/vs9|10/Makefile.am fixed 'make distclean'
but broke 'make -jN dist', so fix that by listing the *.headers and using
that list as a dependency and to remove those files in one single command
right after we generate the gtk-install.vsprops template, so that we don't
have to worry about them in 'make distclean'.

Finnish translation update

app chooser widget: protect against show_all

The visibility of the 'no apps' placeholder is managed by
the dialog, it should not be affected by gtk_widget_show_all.

https://bugzilla.gnome.org/show_bug.cgi?id=748080

app chooser: Avoid duplicates

At the time we populate the model "initially" in constructed(),
it has already been filled and cleared a couple of times (we do
that every time one of the construct properties gets set). So
we can't assume that the model is empty, and have to clear it
first. Otherwise, we add duplicates to the list.

https://bugzilla.gnome.org/show_bug.cgi?id=748080

build: only compile plug/socket when available

Plug and Socket require X11 windowing. Often times this is compiled
on systems with both wayland and x11, but not always. Quartz is an
example where it is usually not compiled.

Updated Serbian translation

3.18.0

Fix distclean

places view: fix a crash in finalize

Fix a GtkPlacesView crash in the test suite, _again_.

flow box: Fix up docs

Remove GtkWidgetAuxInfo from the docs

Updated Serbian Latin translation

Updated Serbian translation

configure: Don't put glib version flags into pc files

Use a different CFLAGS variable to carry them. I chose the DEBUG_FLAGS
because that's essentially what those flags are: debug helpers.

widget-factory: Remove libcanberra support

GTK cannot depend on libcanberra-gtk which depends on GTK. This causes
a circular dependency and is especially neat if installed GTK is
different enough from uninstalled GTK.

Update Czech translation

gdkwindow: Fix list_insert_link_before

We forgot to set prev->next to the new link

gdk: Finish conversion to in-struct GdkWindow list nodes

widget: get_widget_windows() don't modify window child lists

We can't modify the lists returned from gdk_window_peek_children,
instead use get_children().

widget: Make sure we only run controllers on realized widgets

The default event bubbling paths are prone to just running event controllers
even after the widget was potentially unrealized/destroyed in an event
handler callback, so bail out early if that's the case.

https://bugzilla.gnome.org/show_bug.cgi?id=755352

gesture: strengthen against destroyed windows

Bail out when we receive an event with an already destroyed window,
we'll hopefully get reset/cancelled afterwards on grab broken events.

https://bugzilla.gnome.org/show_bug.cgi?id=755352

window: cancel unclaimed sequences after GtkEventController::handle_event.

To GtkGesture machinery, if an event triggers a controller/gesture signal,
and gesture reset/cancellation as a result, the event has been managed
after all.

Commit e3bd895667 effectively changed the return value of the
wrapping gtk_event_controller_handle_event() function, which broke some
paths (eg. gtk_popover_button_press() wouldn't while the GTK+ grab was
active for this reason because the button press event was consumed early
on gtk_window_check_handle_wm_event()).

That patch is not too off-track given potential child widgets' behavior,
we want nonetheless to distinguish the denied vs cancelled paths here
(because GtkWindow itself relies on the GtkGesture behavior described in
the first paragraph on the begin_move/resize paths), so just reset
gestures after the event has already gone through the GtkEventController
so the return value is unaffected.

Updated Kazakh translation

MSVC Builds: Update Project GUIDs

This updates the Visual Studio Project GUIDs so that they don't repeat with
the GTK+-2.24.x ones, as the 3.x projects can be used with the 2.24.x in a
all-in-one solution file (such as when one wants to use a complete GTK+2
and GTK+3 stack when porting Windows applications from GTK+2 to GTK+3), and
each project in a solution file is expected to have an unique GUID.

Updated Norwegian bokmål translation.

Updated Danish translation

Updated Danish translation

Updated Latvian translation

Updated Swedish translation

test-simplify: Explicitly use bash

The script uses bash features, after all.

https://bugzilla.gnome.org/show_bug.cgi?id=755274

Updated Japanese translation

Updated Danish translation

Updated Swedish translation

Updated Greek translation

Updated Japanese translation

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation

gtkdemo: Plug leaks in listbox demo

Updated Danish translation

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation

Updated Russian translation

Updated Russian translation

Revert "gtkwidget: Ensure unrealization during event dispatching cancels gestures"

This reverts commit 13873d2c507783042df55ad2c48f37a346121fd0.

Revert "widget: Fix propagation of gesture cancellation on widget unrealize/destroy"

This reverts commit 3aaf730901d04a3e4a102cb95e769a48ec5027e3.

Revert "widget: Cancel also denied sequences"

This reverts commit 63e255e7b5326888e83bf84f1147012938c81000.

window: Reset on unhandled gestures right away

Traditionally a sequence is set to GTK_EVENT_SEQUENCE_DENIED state when
it is to be ignored, which means it is dormant, but still managed by the
gesture (accounting, "denied" sequences still make "slots" in multitouch
gesture busy, etc...).

This gesture will run for all button presses and releases in the window
though when presses happen on the "window content" region, and we can't
account for every children to be as educated as setting the proper mask
on every window, or ensuring events will be propagated as they should.

In order to cater for this, just reset the gestures, we can live without
such accounting in these specific GtkGestureSingle gestures.

https://bugzilla.gnome.org/show_bug.cgi?id=754098

gdk: Mark window surfaces as dirty before reading from them

We do not know what happened to this surface outside of GDK.
Especially for foreign windows, they will have been modified
by external applications.

So be on the safe side and tell Cairo to clear all its caches.

https://bugzilla.gnome.org/show_bug.cgi?id=754952

Updated Turkish translation

Updated Lithuanian translation

Updated Hungarian translation

MSVC Builds: Make Re-use Comment in Autotools Module Clearer

pixelcache: use factor instead of margin

Instead of using a fixed size, use a factor of the surface size. This
helps in situations where animations of surrounding widgets are used
and cause a rapid rate of surface destroy/create cycles.

Updated French translation

wayland: Avoid running stale cursor animation timeouts

gdk_wayland_device_update_window_cursor() is inconsistently returning
TRUE/FALSE, despite the timeout being always replaced for new cursor
frames. This could end up in these timeouts being "leaked" and running
as long as the window has an animated cursor.

Fix this by making it really sure we return G_SOURCE_REMOVE, although
now we keep track of animation delays, so the timeout will be reused
for constant time animations.

widget: Cancel also denied sequences

It makes no sense to skip denied sequences here, the gestures are
still carrying out the accounting for these, which must be also put
to an end if we're possibly not receiving any further events from
this sequence.

https://bugzilla.gnome.org/show_bug.cgi?id=754098

widget: Fix propagation of gesture cancellation on widget unrealize/destroy

At the time event_check_cancel_sequence_on_hierarchy() is called, the widget
has been already unparented. Given the widget itself is being destroyed,
cancellation on it is impending in one way or another, we still must
propagate cancellation across all parents, so retrieve it early before
possible widget destruction.

https://bugzilla.gnome.org/show_bug.cgi?id=754098

scrolledwindow: Set the scrollbar as "over" immediately during slider grabs

Otherwise it's attempted through a timeout, which gets cancelled early after,
and the slider disappears after a while with no mouse activity despite the
ongoing implicit grab.

Once the grab is finished, check_update_scrollbar_proximity() will be called
again on both scrollbars, and the fade out animation will be triggered as a
result.

https://bugzilla.gnome.org/show_bug.cgi?id=754745

file chooser: Avoid a crash

When right-clicking in an empty folder, you should get a context
menu, not a crash. The code for positioning the popover was not
handling the eventuality of no row under the pointer. Just position
the popover right at the click location in this case.

https://bugzilla.gnome.org/show_bug.cgi?id=755021

widget path: Don't leak sibling paths

Plug another memory leak

Plug a memory leak

Update Persian translation

Update Persian translation

css shadows: Cache blured masks for the corners of outset shadows

A lot of time was spend rendering the shadows on windows with CSD, in
particular the corner pieces, since they are the largest parts. This
patch catches this particular case and caches the pre-rendered blur
masks.

This makes the shadow code go from 25% to 8% of the time when resizing
gtk3-demo.

css shadows: Break out side & corner shadows into helper functions

This changes nothing, but makes the code a bit easier to read

ui: Remove border from dialogs again